-
Notifications
You must be signed in to change notification settings - Fork 619
Version Packages #7231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Version Packages #7231
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
|
""" WalkthroughThe changes remove the previously added Changes
Suggested labels
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (8)
💤 Files with no reviewable changes (4)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (3)
⏰ Context from checks skipped due to timeout of 90000ms (8)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
packages/wagmi-adapter/CHANGELOG.md (1)
3-3: Clarify empty release notes
Since there are no adapter-specific changes in this patch, consider adding a short note (e.g., “No changes; version alignment only”) under the## 0.2.89header to inform consumers.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
.changeset/loose-tools-move.md(0 hunks)packages/thirdweb/CHANGELOG.md(1 hunks)packages/thirdweb/package.json(1 hunks)packages/wagmi-adapter/CHANGELOG.md(1 hunks)packages/wagmi-adapter/package.json(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/loose-tools-move.md
⏰ Context from checks skipped due to timeout of 90000ms (8)
- GitHub Check: Unit Tests
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Size
- GitHub Check: E2E Tests (pnpm, webpack)
- GitHub Check: E2E Tests (pnpm, vite)
- GitHub Check: Build Packages
- GitHub Check: Lint Packages
- GitHub Check: Analyze (javascript)
🔇 Additional comments (5)
packages/thirdweb/package.json (1)
3-3: Patch version bump looks correct
The version was updated from5.102.2to5.102.3to ship the PayEmbed token-pricing fix. Ensure theCHANGELOG.mdentry aligns with this bump and the release automation will publish the correct version.packages/thirdweb/CHANGELOG.md (2)
3-6: Changelog header for v5.102.3 is correctly added
The## 5.102.3release header and its### Patch Changessubsection accurately document the new patch version in alignment with thepackage.jsonbump.
7-7: Patch entry accurately reflects fix details
The entry for PR #7230 with commit8245c06and credit to @gregfromstl clearly describes the token pricing fix in PayEmbed.packages/wagmi-adapter/package.json (1)
3-3: Version bump is correct
The package version has been incremented from0.2.88to0.2.89in line with the release automation. No other metadata or dependency changes are needed here.packages/wagmi-adapter/CHANGELOG.md (1)
3-3: Changelog header added correctly
A new## 0.2.89entry has been inserted above0.2.88, matching the package version bump.
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7231 +/- ##
==========================================
- Coverage 55.62% 55.60% -0.03%
==========================================
Files 908 908
Lines 58586 58552 -34
Branches 4138 4132 -6
==========================================
- Hits 32589 32555 -34
Misses 25891 25891
Partials 106 106
🚀 New features to boost your workflow:
|
ddac2e5 to
375ec79
Compare
375ec79 to
73aa3a7
Compare
73aa3a7 to
3893edc
Compare
3893edc to
58ea417
Compare
58ea417 to
b3f1158
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
[email protected]
Patch Changes
#7240
ad8cc9aThanks @joaquim-verges! - AddBridge.tokens()function to retrieve supported Universal Bridge tokensNew function allows fetching and filtering tokens supported by the Universal Bridge service. Supports filtering by chain ID, token address, symbol, name, and includes pagination with limit/offset parameters.
#7242
f10fbc8Thanks @joaquim-verges! - Optimize fiat conversion query#7230
8245c06Thanks @gregfromstl! - Fixes token pricing in PayEmbed#7234
4e93539Thanks @MananTank! - Fallback to insight response if RPC request fails in ERC721 & ERC1155getNFTsextension#7241
38627d3Thanks @joaquim-verges! - Handle smart account detection for inApp and ecosystem wallets@thirdweb-dev/[email protected]
PR-Codex overview
This PR primarily focuses on updating version numbers and enhancing the
Bridgeclass in thethirdwebpackage. It introduces a new function to retrieve and filter supported Universal Bridge tokens, along with several optimizations and bug fixes.Detailed summary
versioninpackages/thirdweb/package.jsonto5.102.3.versioninpackages/wagmi-adapter/package.jsonto0.2.89.Bridge.tokens()function for retrieving and filtering Universal Bridge tokens.PayEmbed.getNFTsextension to fallback on insight response if RPC request fails.Summary by CodeRabbit